Skip to content

Conversation

ronjalh
Copy link
Member

@ronjalh ronjalh commented Nov 10, 2026

Summary

  • Drawings are shown at most 400 px wide (the size of the files); before, up to 1812 px on a 1920 px screen.
  • width and height on both <img>, so the space is reserved before loading.
  • Pages, sizes and bytes written in docs/SUSTAINABILITY.md. No srcset: the files are small (6.7 kB on average) and only one size exists.

Measured in a real browser (375-1920 px). Frontend tests pass (685), tsc, eslint, prettier clean. Draft: not reviewed yet.

Closes #174

🤖 Generated with Claude Code

@ronjalh ronjalh self-assigned this Nov 10, 2026
@ronjalh ronjalh marked this pull request as ready for review November 11, 2026 06:10
@ronjalh ronjalh requested a review from a team as a code owner November 11, 2026 06:10
@ronjalh ronjalh marked this pull request as draft November 11, 2026 06:25
@ronjalh ronjalh marked this pull request as ready for review November 11, 2026 17:11
@ronjalh
Copy link
Member Author

ronjalh commented Nov 11, 2026 •

I'm not sure I agree with this peer review, what are your thoughts?

The peer review:
Til slutt strekkes bildene på detaljsiden langt over sin egentlige størrelse, og derfor blir de uskarpe. Med en maksbredde på bildet, eller bilde og info i to kolonner, slik som i mockupen deres, ville det sett skarpere ut og utnyttet bred skjerm bedre. (samme fra annen) Elementtegningene er 400 px brede, men vises opptil 1260 px brede på elementsiden og blir tydelig uskarpe.

Now:
image

Before:
image

Sign in to join this conversation on GitHub.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant